Tell, don't ask
より引用
よくない
code:s.html.erb
<% if current_user.admin? %>
<%= current_user.admin_welcome_message %>
<% else %>
<%= current_user.user_welcome_message %>
<% end %>
ベター
code:s.html.erb
<%= current_user.welcome_message %>
if文(分岐)があるようなものは、 メソッドに押し込めて、 表側はシンプルに(呼び出すように)。 ということっぽい